Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PoC AGS dev container #5026

Merged
merged 11 commits into from
Jan 15, 2025
Merged

Conversation

JohanForngren
Copy link
Contributor

@JohanForngren JohanForngren commented Jan 13, 2025

Why are these changes needed?

This is the first draft of a dev container for AGS. A dev container would streamline contributor onboarding and eliminate environment inconsistencies, reducing setup friction and "works on my machine" issues. This is crucial for AutoGen Studio's Python/React architecture, which requires multiple runtime environments. See #5025

Related issue number

Depends on #5008
Closes #5025

Checks

Signed-off-by: Johan Forngren <[email protected]>

Update uv.lock with tiktoken dependency from fbfdc9f

Signed-off-by: Johan Forngren <[email protected]>
Signed-off-by: Johan Forngren <[email protected]>
…tallation and improved performance. Removed partial hot reload setup, as it required bigger changes to the project. Improved documentation, and fixed markdown lint issues.

Signed-off-by: Johan Forngren <[email protected]>
…allation methods and improving formatting. Adjusted notes for Windows users and corrected minor grammatical issues.

Signed-off-by: Johan Forngren <[email protected]>
@JohanForngren JohanForngren changed the title PoC dev container PoC AGS dev container Jan 13, 2025
Copy link

codecov bot commented Jan 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.07%. Comparing base (6954e51) to head (5fcf490).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5026      +/-   ##
==========================================
- Coverage   69.07%   69.07%   -0.01%     
==========================================
  Files         159      159              
  Lines       10346    10346              
==========================================
- Hits         7147     7146       -1     
- Misses       3199     3200       +1     
Flag Coverage Δ
unittests 69.07% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@victordibia
Copy link
Collaborator

Thanks for this @JohanForngren ,
I am sure it will help others get started with running AGS locally.
Left some minor comments.

JohanForngren and others added 4 commits January 14, 2025 11:50
…blocks separated only by a blank line to avoid MD028 - Blank line inside blockquote.

Signed-off-by: Johan Forngren <[email protected]>
Signed-off-by: Johan Forngren <[email protected]>
@victordibia victordibia merged commit 8efe0c4 into microsoft:main Jan 15, 2025
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dev container for AutoGen Studio
2 participants